From: Jonas Ådahl <jadahl@gmail.com>
Date: Fri, 4 Mar 2022 21:52:29 +0000 (+0100)
Subject: workspaceThumbnail: Use transient object for connecting some signals
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~516^2^2~11
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=b3c7a855c65d6046789947ee4cfc0c649b399d75;p=gnome-shell.git

workspaceThumbnail: Use transient object for connecting some signals

We want to connect and disconnect a bunch of signals when showing/hiding
the worskpace thumbnails. To make this easy, use a transient object that
we tie to these signal handlers.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2220>
---

diff --git a/js/ui/workspaceThumbnail.js b/js/ui/workspaceThumbnail.js
index cb8a19188..af94a9d06 100644
--- a/js/ui/workspaceThumbnail.js
+++ b/js/ui/workspaceThumbnail.js
@@ -5,6 +5,7 @@ const { Clutter, Gio, GLib, GObject, Graphene, Meta, Shell, St } = imports.gi;
 
 const DND = imports.ui.dnd;
 const Main = imports.ui.main;
+const { TransientSignalHolder } = imports.misc.signalTracker;
 const Util = imports.misc.util;
 const Workspace = imports.ui.workspace;
 
@@ -949,6 +950,7 @@ var ThumbnailsBox = GObject.registerClass({
             return;
 
         const { workspaceManager } = global;
+        this._transientSignalHolder = new TransientSignalHolder(this);
         workspaceManager.connectObject(
             'notify::n-workspaces', this._workspacesChanged.bind(this),
             'active-workspace-changed', () => this._updateIndicator(),
@@ -957,9 +959,9 @@ var ThumbnailsBox = GObject.registerClass({
                     return a.metaWorkspace.index() - b.metaWorkspace.index();
                 });
                 this.queue_relayout();
-            }, this);
+            }, this._transientSignalHolder);
         Main.overview.connectObject('windows-restacked',
-            this._syncStacking.bind(this), this);
+            this._syncStacking.bind(this), this._transientSignalHolder);
 
         this._targetScale = 0;
         this._scale = 0;
@@ -979,8 +981,8 @@ var ThumbnailsBox = GObject.registerClass({
         if (this._thumbnails.length == 0)
             return;
 
-        global.workspace_manager.disconnectObject(this);
-        Main.overview.disconnectObject(this);
+        this._transientSignalHolder.destroy();
+        delete this._transientSignalHolder;
 
         for (let w = 0; w < this._thumbnails.length; w++)
             this._thumbnails[w].destroy();